-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Makefile: Install and library targets and configurable BUILDDIR #2
Conversation
zuiderkwast
commented
Nov 2, 2023
•
edited
Loading
edited
- Configurable BUILDDIR
- Add install and library targets
- Remove unnecessary variables
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
9f3bf73
to
27b2064
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WDYT about renaming SOLIBNAME to LIBNAME, and SOLIBNAME to LIBNAME?
i.e
LIBNAME = liburl_parser
SONAME ?= $(LIBNAME).$(SOEXT).$(SOMAJOR).$(SOMINOR)
SOLIBNAME= $(LIBNAME).$(SOEXT).$(SOMAJOR).$(SOMINOR).$(SOREV)
Then the LIBNAME can additionally be used for the static library.. or maybe we divert to much from the original then?
Yes we can. It makes sense.
Yes, that's why i didn't do the change. But it doesn't matter much. It's more important than the C code is as close as possible to the original. I'll do the change. |
Looks better? Do you want a variable for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm happy!